ci(homebrew): enable test coverage on Linux #3842
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3842 +/- ##
==========================================
+ Coverage 14.26% 15.27% +1.01%
==========================================
Files 89 91 +2
Lines 18906 18971 +65
Branches 8662 8702 +40
==========================================
+ Hits 2696 2898 +202
- Misses 14439 15013 +574
+ Partials 1771 1060 -711
Flags with carried forward coverage won't be shown. Click here to find out more. |
ReenigneArcher
force-pushed
the
ci/homebrew/enable-test-coverage
branch
11 times, most recently
from
09350b4 to
bccfda2
Compare
Bundle ReportBundle size has no change ✅ |
ReenigneArcher
force-pushed
the
ci/homebrew/enable-test-coverage
branch
13 times, most recently
from
52eb7ae to
6cf5e8a
Compare
Member
|
It looks like this PR has been idle for 90 days. If it's still something you're working on or would like to pursue, please leave a comment or update your branch. Otherwise, we'll be closing this PR in 10 days to reduce our backlog. Thanks! |
ReenigneArcher
force-pushed
the
ci/homebrew/enable-test-coverage
branch
from
6cf5e8a to
5772a12
Compare
ReenigneArcher
force-pushed
the
ci/homebrew/enable-test-coverage
branch
from
ac5a86a to
c88af19
Compare
ReenigneArcher
force-pushed
the
ci/homebrew/enable-test-coverage
branch
9 times, most recently
from
ab5edc0 to
3f3e2da
Compare
ReenigneArcher
force-pushed
the
ci/homebrew/enable-test-coverage
branch
from
3f3e2da to
0dd02ce
Compare
ReenigneArcher
force-pushed
the
ci/homebrew/enable-test-coverage
branch
from
0dd02ce to
30b9f34
Compare
ReenigneArcher
force-pushed
the
ci/homebrew/enable-test-coverage
branch
3 times, most recently
from
048eb2f to
878d8f0
Compare
ReenigneArcher
force-pushed
the
ci/homebrew/enable-test-coverage
branch
2 times, most recently
from
d427f0d to
e7524b6
Compare
GCC dependency and related environment setup are now only applied on Linux, aligning with platform-specific requirements. The gcovr coverage generation and assertion are also restricted to Linux.
Adds a 'coverage' flag to each job in the CI matrix and updates the upload step to only run when coverage is enabled for that job. This allows more granular control over which environments generate and upload coverage reports.
ReenigneArcher
force-pushed
the
ci/homebrew/enable-test-coverage
branch
from
9bada82 to
52629c6
Compare
ReenigneArcher
changed the title
|
Please retry analysis of this Pull-Request directly on SonarQube Cloud |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Now that tmp files are retained from homebrew build and test steps, the gcno files should remain intact.
Depends on gcovr/gcovr#1047
TODO:
Screenshot
Issues Fixed or Closed
Type of Change
.github/...)Checklist